Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace unmaintained actions-rs/toolchain action in CI workflows #58

Merged
merged 1 commit into from
Apr 23, 2023
Merged

Replace unmaintained actions-rs/toolchain action in CI workflows #58

merged 1 commit into from
Apr 23, 2023

Conversation

striezel
Copy link
Contributor

Basically all of the actions-rs/* actions are unmaintained. See actions-rs/toolchain#216 for more information. Due to their age they generate several warnings in CI runs, for example in https://github.com/NiklasEi/bevy_game_template/actions/runs/4772937853:

Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions-rs/toolchain@v1. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

To get rid of those warnings the occurrences of actions-rs/toolchain are replaced by dtolnay/rust-toolchain.

Basically all of the `actions-rs/*` actions are unmaintained. See
<actions-rs/toolchain#216> for more
information. Due to their age they generate several warnings in
CI runs.

To get rid of those warnings the occurrences of
`actions-rs/toolchain` are replaced by `dtolnay/rust-toolchain`.
@janhohenheim
Copy link
Contributor

Just a little PSA: the toolchain you use now ignores rust toolchain files. It doesn't matter in this case, but something to keep in mind :)

@NiklasEi NiklasEi merged commit a012672 into NiklasEi:main Apr 23, 2023
@NiklasEi
Copy link
Owner

Just a little PSA: the toolchain you use now ignores rust toolchain files. It doesn't matter in this case, but something to keep in mind :)

This just reminded me of your ping in janhohenheim/foxtrot#249, which I had entirely forgotten to answer on 🙈
Thanks for the heads-up! I haven't used toolchain files before, but it's good to know, should they ever come up.

@janhohenheim
Copy link
Contributor

@NiklasEi hehe yeah, I thought you might recognize this PR from somewhere 😉

@striezel striezel deleted the actions-rs-toolchain-replacement branch April 23, 2023 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants